Skip to content

style(docs): add subtle shadow to screenshots#14184

Merged
skjnldsv merged 2 commits intomasterfrom
fix/noid/screenshots-shadow
Apr 17, 2026
Merged

style(docs): add subtle shadow to screenshots#14184
skjnldsv merged 2 commits intomasterfrom
fix/noid/screenshots-shadow

Conversation

@Antreesy
Copy link
Copy Markdown
Contributor

@Antreesy Antreesy commented Apr 1, 2026

☑️ Resolves

  • Fix blending of light-theme screenshots with the background
  • Not visible on dark-theme docs, but it's a step forward nevertheless

🖼️ Screenshots

Before After
image image
image image

Copy link
Copy Markdown
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! :) Don’t we use variables here as well tho?
It’s fine to also use some bigger border-radius for images here, could be our border-radius-large: 10px;

@Antreesy
Copy link
Copy Markdown
Contributor Author

Antreesy commented Apr 1, 2026

Don’t we use variables here as well?

I see only hardcoded values, so guess not. Adjusted, added a dark-theme shadow as well

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the fix/noid/screenshots-shadow branch from 32a4df4 to 74f865c Compare April 1, 2026 09:46
@skjnldsv skjnldsv enabled auto-merge April 17, 2026 05:02
@skjnldsv skjnldsv merged commit 359e49f into master Apr 17, 2026
17 checks passed
@skjnldsv skjnldsv deleted the fix/noid/screenshots-shadow branch April 17, 2026 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants